Add Cert Manager to Operator #70
Conversation
docs/provided-mode-test.md
Outdated
| @@ -0,0 +1,445 @@ | |||
| # Provided TLS mode: step-by-step test (AKV + Secrets Store CSI) | |||
There was a problem hiding this comment.
Lets put all tls setup doc under our /documentdb-playground/tls folder.
| @@ -0,0 +1,186 @@ | |||
| #!/usr/bin/env bash | |||
There was a problem hiding this comment.
Lets move all the TLS cert-manager scripts to the documentdb-playground/tls folder.
| done | ||
|
|
||
| if [[ -z "$RESOURCE_GROUP" ]]; then | ||
| RESOURCE_GROUP="guanzhou-${SUFFIX}-rg" |
There was a problem hiding this comment.
We have guanzhou hardcoded in many places. Please replace them with param or documentdb
|
We need a delete cluster or cleanup command to avoid cost. |
|
Here are my thoughts. Here we'll have a README or tls-setup.md file, which will give two simple commands maybe.
|
| @@ -0,0 +1,657 @@ | |||
| #!/bin/bash | |||
There was a problem hiding this comment.
Create a new folder tls under /documentdb-playground and move all the scripts and docs under it.
| @@ -17,10 +17,10 @@ walReplica: false # Set to true to deploy the WAL replica plugin | |||
|
|
|||
| image: | |||
| documentdbk8soperator: | |||
| repository: ghcr.io/microsoft/documentdb-kubernetes-operator/operator | |||
| repository: ghcr.io/guanzhousongmicrosoft/documentdb-kubernetes-operator/operator | |||
There was a problem hiding this comment.
Keep the official registry name.
| pullPolicy: Always | ||
| sidecarinjector: | ||
| repository: ghcr.io/microsoft/documentdb-kubernetes-operator/sidecar | ||
| repository: ghcr.io/guanzhousongmicrosoft/documentdb-kubernetes-operator/sidecar |
There was a problem hiding this comment.
Same. Keep official repo name.
| Enabled: pointer.Bool(true), | ||
| Parameters: params, | ||
| }} | ||
| }(), |
There was a problem hiding this comment.
Please rebase. We changed the directory structure.
| #set -e # Exit on any error | ||
|
|
||
| # Configuration | ||
| CLUSTER_NAME="guanzhou-102001" |
There was a problem hiding this comment.
Use documentdb- prefix instead of our usernames.
| # DocumentDB Operator Configuration | ||
| # For testing: use hossain-rayhan/documentdb-operator (fork with Azure enhancements) | ||
| # For production: use microsoft/documentdb-operator (official) | ||
| OPERATOR_GITHUB_ORG="guanzhousongmicrosoft" |
There was a problem hiding this comment.
The default should be microsoft.
docs/provided-mode-test.md
Outdated
| @@ -0,0 +1,445 @@ | |||
| # Provided TLS mode: step-by-step test (AKV + Secrets Store CSI) | |||
3 participants
No description provided.